Conversation
yurii-litvinov
left a comment
yurii-litvinov
left a comment
There was a problem hiding this comment.
Тестов нет, а по условию нужны были
| Microsoft Visual Studio Solution File, Format Version 12.00 | ||
| Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Test1.1", "Test1.1\Test1.1.csproj", "{E451040C-44C5-427F-9CF7-F83F90E54DFF}" | ||
| EndProject | ||
| Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Test1.1.Tests", "Test1.1.Tests\Test1.1.Tests.csproj", "{E628CC17-DD0A-4658-8561-CB4C496A798D}" |
There was a problem hiding this comment.
Даже CI не одобряет отсутствия тестов
| /// </summary> | ||
| public class Client | ||
| { | ||
| private TcpClient _client; |
There was a problem hiding this comment.
Если хоть одно поле у класса IDisposable, то и сам класс должен быть IDisposable, и у себя в Dispose вызывать Dispose всех своих IDisposable-полей. Иначе есть риск, что Dispose у них не вызовется до самого конца программы (что сродни утечке памяти).
|
|
||
| public Client(string host, int port) | ||
| { | ||
| _client = new TcpClient(host, port); |
There was a problem hiding this comment.
Это сразу инициирует подключение, и если хост не ответит, приведёт к ожиданию таймаута, который в случае с TCP может быть довольно большим. И всё это время вызывающий будет заблокирован. Надо было использовать ConnectAsync, и не в конструкторе, а при старте.
| await Task.Run(async () => | ||
| { | ||
| var writer = new StreamWriter(stream) { AutoFlush = true }; | ||
| while (true) |
There was a problem hiding this comment.
Итого у Вас три (!) вложенных бесконечных цикла, тогда как работает (и нужен) только один
| Task.Run(async () => | ||
| { | ||
| while (true) { | ||
| await Writer(_client.GetStream()); |
There was a problem hiding this comment.
Не-а, так Reader будет заблокирован. Надо было Task.WhenAny
| } | ||
| else | ||
| { | ||
| var client = new Client(args[0], Int32.Parse(args[1])); |
There was a problem hiding this comment.
Никто не обещал, что будет хоть один аргумент, кстати
| public class Server | ||
| { | ||
| private int _port; | ||
| private readonly CancellationTokenSource _cancellationToken = new CancellationTokenSource(); |
There was a problem hiding this comment.
| private readonly CancellationTokenSource _cancellationToken = new CancellationTokenSource(); | |
| private readonly CancellationTokenSource _cancellationToken = new (); |
Вы ж не на Java пишете :)
| } | ||
| } | ||
| }); | ||
| } |
There was a problem hiding this comment.
Что-то копипаст, такое в клиенте уже было
Test1.1/Test1.1/Program.cs
Outdated
| else | ||
| { | ||
| var client = new Client(args[0], Int32.Parse(args[1])); | ||
| client.Working(); |
There was a problem hiding this comment.
await server.Working(); но блокирующий client.Working();, несимметрично и неправильно (клиент тоже может быть асинхронным, конечно)
| Console.WriteLine(data + "\n"); | ||
| if (data == "exit") | ||
| { | ||
| StopServer(); |
There was a problem hiding this comment.
Выйти из чата может только сервер, кажется :)
3 participants
No description provided.